Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CS2103-T11-4] TrAcker #174

Open
wants to merge 423 commits into
base: master
Choose a base branch
from
Open

[CS2103-T11-4] TrAcker #174

wants to merge 423 commits into from

Conversation

yyccbb
Copy link

@yyccbb yyccbb commented Mar 7, 2024

TrAcker is an application that aims to help Head Teaching Assistants (Head TAs) track the records of other Teaching Assistants (TAs) and students.

@codecov Codecov
Copy link

codecov bot commented Mar 7, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 70.99863% with 212 lines in your changes are missing coverage. Please review.

Project coverage is 74.45%. Comparing base (b896eb0) to head (bc58c57).

❗ Current head bc58c57 differs from pull request most recent head 2fdc33a. Consider uploading reports for the commit 2fdc33a to get more accurate results

Files Patch % Lines
src/main/java/seedu/address/ui/WarningWindow.java 0.00% 38 Missing ⚠️
...seedu/address/commons/util/StatefulStringUtil.java 38.46% 32 Missing ⚠️
src/main/java/seedu/address/ui/MainWindow.java 0.00% 31 Missing ⚠️
...seedu/address/logic/parser/StatefulParserUtil.java 59.57% 17 Missing and 2 partials ⚠️
.../address/logic/commands/EditTutTagListCommand.java 71.11% 9 Missing and 4 partials ⚠️
src/main/java/seedu/address/ui/PersonCard.java 0.00% 11 Missing ⚠️
...seedu/address/logic/commands/RemovetagCommand.java 78.72% 3 Missing and 7 partials ⚠️
...s/model/person/FieldContainsKeywordsPredicate.java 30.76% 9 Missing ⚠️
src/main/java/seedu/address/model/tag/Tag.java 74.28% 4 Missing and 5 partials ⚠️
...va/seedu/address/logic/commands/DeleteCommand.java 80.00% 7 Missing ⚠️
... and 13 more
Additional details and impacted files 
@@             Coverage Diff              @@
##             master     #174      +/-   ##
============================================
- Coverage     75.26%   74.45%   -0.81%     
- Complexity      419      627     +208     
============================================
  Files            71       94      +23     
  Lines          1338     2063     +725     
  Branches        126      208      +82     
============================================
+ Hits           1007     1536     +529     
- Misses          301      465     +164     
- Partials         30       62      +32

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

wang-xinrong and others added 29 commits April 1, 2024 17:50
@wang-xinrong
Adjust checkstyle required changes
767e207
@yyccbb
Bug fix: remove dangling tag label
7b5dc76
@wang-xinrong
Merge pull request #83 from yyccbb/master
5ad784e 
Bug fix: remove dangling tag label
@wang-xinrong
Add UniqueTutorialTagList to AddressBook
fa1cb68
@wang-xinrong
Restrict TutorialTag marking
a1c8be4
@wang-xinrong
Add CreateTutTag command
2a10ad3
@wang-xinrong
Merge pull request #82 from wang-xinrong/branch-deletetag-command
897d8b7 
Implement Deletetag Command
@wang-xinrong
Correct past test data
0419f9a
@wang-xinrong
Merge remote-tracking branch 'upstream/master' into branch-tag-restri…
8a67a79 
…ctions
@yyccbb
EditPersonDescriptor: remove type field
a00bcd6
@yongkotaro
Fix GUI bug for Find command
86462c8
@yyccbb
EditCommand: remove the tag field
90d2425
@yyccbb
Merge pull request #91 from yyccbb/edit-person-descriptor
e2ce908 
Remove the tag field from EditCommand and update the EditPersonDescriptor accordingly
@wang-xinrong
Merge pull request #88 from yongkotaro/fix-bugs
6bbdb8e 
Fix GUI bug for Find command
@wang-xinrong
Merge remote-tracking branch 'upstream/master' into branch-tag-restri…
7f4c553 
…ctions
@wongkj12
edit sample data to include ta
e4faaff
@yyccbb
Bug fix: display person in singular form when only 1 found
c2bd416
@yyccbb
Merge pull request #92 from yyccbb/master
8872b2a 
Bug fix: display person in singular form when only 1 found
@wongkj12
Update UG
836c04d
@wongkj12
update filter to AND
9807342
@wongkj12
update parameter description
ed04567
@wongkj12
update text
2d9ca4a
@wongkj12
Update text
fef6ef1
@wongkj12
Update Tagging sections
e27acc3
@wongkj12
Merge pull request #93 from wongkj12/fix-find-command
449672f 
Fix find command
@wongkj12
fix sample data test
c572ead
@wongkj12
Merge pull request #94 from wongkj12/ug-v1.3-1
e466250 
Ug v1.3 ver 1
@wang-xinrong
Merge remote-tracking branch 'upstream/master' into branch-tag-restri…
e68a35f 
…ctions
@wang-xinrong
Add test cases
1d60884
wang-xinrong and others added 30 commits April 15, 2024 21:32
@wang-xinrong
Merge pull request #207 from yongkotaro/fix-UG
f01e966 
Fix Command Summary in UG
@yyccbb
Update test cases
5b594e2
@kaitinghh
Update UG
a907098
@yyccbb
Merge pull request #201 from yyccbb/fix-exception-index-display
0ab95a5 
Fix the incorrect error message
@kaitinghh
Merge pull request #208 from kaitinghh/branch/update-dg
1433084 
Update DG and UG
@yongkotaro
Remove Acknowledgements in DG
680449a
@yyccbb
Update UG and error messages
6235149
@wongkj12
Merge pull request #209 from yongkotaro/update-DG
b6e1842 
Remove Acknowledgements in DG
@yyccbb
Fix checkstyle
bd57079
@wang-xinrong
Update UG
cd467ec
@wang-xinrong
Merge pull request #210 from yyccbb/fix-error-message
a30090d 
Update UG and error messages
@yyccbb
Fix description for email regex
f425103
@yyccbb
Merge pull request #212 from yyccbb/fix-error-message
5bdcd94 
Fix description for email regex
@wang-xinrong
Merge pull request #211 from wang-xinrong/branch-fix-available-bug
dde84e9 
Update UG
@kaitinghh
Update UG
acf67c1
@kaitinghh
Merge branch 'master' into branch/update-ug
22f6e67
@kaitinghh
Merge pull request #213 from kaitinghh/branch/update-ug
aa67657 
Update UG
@yyccbb
Update index.md
35fcced
@wang-xinrong
Merge pull request #214 from yyccbb/master
cc259d8 
Update index.md
@wongkj12
update find
bd9eaf7
@kaitinghh
Update UG
d372043
@yyccbb
Merge pull request #215 from kaitinghh/branch/update-ug-2
539cbc8 
Update UG
@yyccbb
Merge pull request #216 from wongkj12/dg-update-2
a22a8b1 
update find
@kaitinghh
Update UG
a642ea0
@kaitinghh
Update UG
3c81fb2
@yongkotaro
Merge pull request #217 from kaitinghh/branch/update-ug-3
30dcdf3 
Branch/update ug 3
@kaitinghh
Update UG
02bbdff
@yyccbb
DG update
8be0f48
@yongkotaro
Merge pull request #218 from kaitinghh/branch-update/ug3
3ae7f81 
Update UG
@yyccbb
Merge pull request #219 from yyccbb/master
2fdc33a 
DG update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@yyccbb @wang-xinrong @yongkotaro @wongkj12 @kaitinghh